Skip to content

Conversation

hamzaremmal
Copy link
Member

@hamzaremmal hamzaremmal commented Sep 24, 2025

These changes were agreed on during the Core Meeting on September 24th, 2025.

[skip test]
[skip test_windows_fast]
[skip test_sbt]

@hamzaremmal hamzaremmal marked this pull request as ready for review September 27, 2025 11:54
@Test def playJson = projects.playJson.run()
@Test def pprint = projects.pprint.run()
@Test def protoquill = projects.protoquill.run()
//@Test def protoquill = projects.protoquill.run()
Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Any particular reason to exclude? Is it failing to build?

Copy link
Contributor

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

I've just seen a commit message which disabled it - let's add a comment in the source file instead so we'd be clear on why it was temporarily excluded

Copy link
Member Author

Choose a reason for hiding this comment

The reason will be displayed to describe this comment to others. Learn more.

Yes, it fails when we drop the dependencies. I'm not sure I want to wait for another 2+ hours for the CI for a comment I will be removing in my next PR (#23963) anyways.

@hamzaremmal hamzaremmal merged commit b67fcf2 into scala:main Sep 27, 2025
51 checks passed
@hamzaremmal hamzaremmal deleted the drop-deps-cb branch September 27, 2025 14:24
Sign up for free to join this conversation on GitHub. Already have an account? Sign in to comment
Labels
None yet
Projects
None yet
Development

Successfully merging this pull request may close these issues.

2 participants